Skip to content

Airgapped callout fixes#256

Merged
sriramveeraghanta merged 1 commit intomasterfrom
airgapped-callout-update
Apr 1, 2026
Merged

Airgapped callout fixes#256
sriramveeraghanta merged 1 commit intomasterfrom
airgapped-callout-update

Conversation

@danciaclara
Copy link
Copy Markdown
Collaborator

@danciaclara danciaclara commented Apr 1, 2026 •
edited by sriramveeraghanta
Loading

Summary by CodeRabbit

  • Documentation
    • Updated airgapped deployment documentation to clarify contact information regarding exceptions to seat cut-offs, tailored pricing, and licensing options.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
developer-docs Ready Ready Preview, Comment Apr 1, 2026 0:22am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026 •
edited
Loading

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 29d6cde5-c812-46f4-981e-26d1cdeff1db

📥 Commits

Reviewing files that changed from the base of the PR and between 38c3e4a and 46b7809.

📒 Files selected for processing (3)
  • docs/self-hosting/methods/airgapped-edition-kubernetes.md
  • docs/self-hosting/methods/airgapped-edition.md
  • docs/self-hosting/methods/airgapped-requirements.md
📝 Walkthrough

Walkthrough

Updated airgapped deployment notice text across three documentation files. Changed "sales team" to "Sales team" and refined the contact message to include "exceptions to the seat cut-off," "tailored pricing," and "licensing info" instead of prior phrasing about "trials" and "lesser seats."

Changes

Cohort / File(s) Summary
Airgapped Deployment Documentation
docs/self-hosting/methods/airgapped-edition-kubernetes.md, docs/self-hosting/methods/airgapped-edition.md, docs/self-hosting/methods/airgapped-requirements.md		 Updated sales contact callout text: capitalized "Sales team" and replaced messaging about trials and lesser seats with language about seat cut-off exceptions, tailored pricing, and licensing info.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop, skip, and text refine,
Sales team shines with brighter shine,
Licensing words now align,
Clearer paths for those who sign! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updates to airgapped deployment callout text across three documentation files for improved clarity and consistency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch airgapped-callout-update

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sriramveeraghanta sriramveeraghanta merged commit 9fa5d12 into master Apr 1, 2026
7 of 8 checks passed
@sriramveeraghanta sriramveeraghanta deleted the airgapped-callout-update branch April 1, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sriramveeraghanta sriramveeraghanta sriramveeraghanta approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danciaclara @sriramveeraghanta